Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add methods for getting file data from the vfs #1

Merged
merged 7 commits into from
Aug 12, 2024
Merged

Add methods for getting file data from the vfs #1

merged 7 commits into from
Aug 12, 2024

Conversation

CosmicDreamsOfCode
Copy link
Contributor

Only things to really note are that i had to add something to let the method for getting the file data know what encryption to use, probably better like this anyway
and that obviously i can't get any oodle slices since we don't support that right now

@CosmicDreamsOfCode
Copy link
Contributor Author

the combine byte array methood should probably be moved somewhere else or changed to something for blocks if you'd prefer the data be returned as a block, i feel that'll probably be the most contentious bit of this PR

Galanthus/SdfToc.cs Show resolved Hide resolved
Galanthus/SdfToc.cs Outdated Show resolved Hide resolved
Galanthus/SdfToc.cs Outdated Show resolved Hide resolved
Copy link
Owner

@wannkunstbeikor wannkunstbeikor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just some small minor changes, after u fix them im going to merge it thanks a lot

Galanthus/GameManager.cs Outdated Show resolved Hide resolved
Galanthus/SdfToc.cs Outdated Show resolved Hide resolved
Galanthus/SdfToc.cs Outdated Show resolved Hide resolved
GalanthusCli/Program.cs Outdated Show resolved Hide resolved
@wannkunstbeikor wannkunstbeikor merged commit 8259692 into wannkunstbeikor:master Aug 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants